Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Form to provide metadata : Headquarters, unit , currency , company name, #74

Merged
merged 15 commits into from
Apr 24, 2024
Merged

Form to provide metadata : Headquarters, unit , currency , company name, #74

merged 15 commits into from
Apr 24, 2024

Conversation

jeremyfix
Copy link
Collaborator

@jeremyfix jeremyfix commented Apr 22, 2024 •
edited
Loading

Cette PR introduit une nouvelle page pour donner différentes informations concernant le rapport :

image

La page est introduite en Step 2, tout de suite après l'import du PDF , parce que la suite du traitement ne concerne que les tables; J'ai préféré donc régler les questions des metadata juste après la sélection des pages pertinentes.

Les metadata sont ajoutées comme des colonnes de valeur constante dans le fichier csv exporté à la dernière étape;

image

close #59

@jeremyfix jeremyfix marked this pull request as ready for review April 22, 2024 20:39
@RonanMorgan
Copy link
Contributor

ah I just found an issue with the last page
metadata

@jeremyfix
Copy link
Collaborator Author

ah I just found an issue with the last page metadata

Je pense que ça ne doit arriver que si tu sautes aux pages suivantes , sans avoir cliqué sur le "Submit" de la page "Metadata" . C'est bien ça que tu avais fait ? Parce que la clé "metadata" du st.session_state est créé sur la page 2_Metadata.py :

12_taxobservatory/app/pages/2_Metadata.py

Lines 92 to 100 in e6df58c

if submitted:
st.session_state["metadata"] = {
"company_name": company_name,
"sector": sector,
"year": year,
"currency": currency,
"unit": unit,
"headquarter": headquarter,
}

@jeremyfix
Copy link
Collaborator Author

L'erreur était levée dans to_csv_file qui supposait l'existence de la clé metadata , ce qui supposait qu'on avait forcément cliqué sur "Submit" sur la page de metadata ; J'ai ajouté une condition pour laisser ces valeurs vides dans l'export csv si jamais la page a été passée.

@jeremyfix jeremyfix merged commit 388b7a1 into main Apr 24, 2024
2 checks passed
@jeremyfix jeremyfix deleted the streamlit_issue_59 branch April 24, 2024 06:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@RonanMorgan RonanMorgan RonanMorgan approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Streamlit : Ajout de métadata currency, unit, year, HQ
2 participants
@jeremyfix @RonanMorgan